Skip to content

[tarfile] Use Literal for TarFile.{debug,errorlevel}#15368

Merged
srittau merged 3 commits intopython:mainfrom
srittau:tarfile-debug
Mar 11, 2026
Merged

[tarfile] Use Literal for TarFile.{debug,errorlevel}#15368
srittau merged 3 commits intopython:mainfrom
srittau:tarfile-debug

Conversation

@srittau
Copy link
Collaborator

@srittau srittau commented Feb 4, 2026

@github-actions

This comment has been minimized.

Copy link
Contributor

@donbarbos donbarbos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Have you considered updating these params for others TarFile methods: __init__ method for the rest versions, and …open methods, which seem to simply forward **kwargs to the constructor?

@github-actions

This comment has been minimized.

@srittau
Copy link
Collaborator Author

srittau commented Mar 11, 2026

LGTM. Have you considered updating these params for others TarFile methods: __init__ method for the rest versions, and …open methods, which seem to simply forward **kwargs to the constructor?

Weird that I didn't do that at least for the other __init__ branch. Anyway, that's done now for all debug and errorlevel arguments if you want to have another look.

@github-actions
Copy link
Contributor

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

@srittau srittau merged commit 25438e5 into python:main Mar 11, 2026
51 checks passed
@srittau srittau deleted the tarfile-debug branch March 11, 2026 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants